Skip to content

ci: vendor validation scripts and remove remote action pins - #86

Merged
hyperpolymath merged 1 commit into
mainfrom
fix-ci-actions
Jul 27, 2026
Merged

ci: vendor validation scripts and remove remote action pins#86
hyperpolymath merged 1 commit into
mainfrom
fix-ci-actions

Conversation

@hyperpolymath

@hyperpolymath hyperpolymath commented Jul 27, 2026

Copy link
Copy Markdown
Owner

Automated PR to fix CI after deleted actions.


Summary by Gitar

  • CI Workflows:
    • Added .githooks/validate-a2ml.sh and .githooks/validate-k9.sh scripts for local and CI validation
    • Updated .github/workflows/dogfood-gate.yml to run vendored scripts instead of remote actions

This will update automatically on new commits.

@gitar-bot

gitar-bot Bot commented Jul 27, 2026

Copy link
Copy Markdown
CI failed: K9 contract and A2ML manifest validations failed due to missing required metadata fields and magic numbers in contract/manifest files.

Overview

Validation checks failed across two separate CI jobs: K9 contract validation failed due to missing magic numbers and pedigree metadata fields, and A2ML manifest validation failed due to a missing required identity field in one or more .a2ml files.

Failures

K9 Contract Validation Failure (confidence: high)

  • Type: test
  • Affected jobs: 90125712239
  • Related to change: yes
  • Root cause: The K9 contract files lack the required 'K9!' magic number on the first non-empty line and are missing the 'name' field in their pedigree block.
  • Suggested fix: Update the K9 contract files so that the first non-empty line is exactly 'K9!' and include the required 'name' field under the pedigree metadata block.

A2ML Manifest Validation Failure (confidence: high)

  • Type: test
  • Affected jobs: 90125712142
  • Related to change: yes
  • Root cause: An .a2ml manifest file failed validation in .githooks/validate-a2ml.sh because it is missing a required identity field (agent-id, name, or project).
  • Suggested fix: Review the modified or added .a2ml manifest files and ensure all required identity fields are properly populated.

Summary

  • Change-related failures: 2 validation failures (K9 contracts and A2ML manifests)
  • Infrastructure/flaky failures: 0
  • Recommended action: Update the K9 contract and A2ML manifest files to satisfy the validation script requirements.
Code Review ✅ Approved

Vendors validation scripts into .githooks/ and updates the dogfood gate workflow to execute them locally instead of relying on remote actions. No issues found.

Auto-approved: No blocking issues found.
Please see Auto-approve Docs for details on setting custom approval criteria.

Tip

Comment Gitar fix CI or enable auto-apply: gitar auto-apply:on

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@gitar-bot gitar-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gitar has auto-approved this PR (configure)

@hyperpolymath
hyperpolymath merged commit 97ae799 into main Jul 27, 2026
30 of 33 checks passed
@hyperpolymath
hyperpolymath deleted the fix-ci-actions branch July 27, 2026 23:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant